Skip to content

Add elixir support #51

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

arthurcogo
Copy link

@arthurcogo arthurcogo commented Apr 8, 2025

Elixir queries based off this source: https://elixir-lang.org/tree-sitter-elixir/

@arthurcogo
Copy link
Author

WIP... but I'm not sure how to test things properly. I'm getting spelling errors showing in Zed for the /examples/example.ex file - but I'm also getting the same if I comment out everything in the elixir.scm file! Either way it is also identifying things like defmodule as a spelling error when it shouldn't.

Will keep working on this, but any support or guidance welcomed!

PS Great docs on how to add a new language, otherwise would never have made it this far!

@blopker
Copy link
Owner

blopker commented Apr 11, 2025

Thanks for the work on this! The issue is likely because the Zed plugin (https://github.com/blopker/codebook-zed) needs to be updated to know about Elixir. That being said, if you add tests to crates/codebook/tests it will catch any issues without updating the plugin.

Once you 1) add a test_elixir.rs file to the previously mentioned test folder and 2) add an example file to the examples folder, I'll be able to update the plugin and test it locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants